Skip to content
This repository has been archived by the owner on Dec 31, 2024. It is now read-only.

Update ImapClient.cs #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update ImapClient.cs #119

wants to merge 1 commit into from

Conversation

pocky6
Copy link

@pocky6 pocky6 commented Sep 18, 2015

I'm French (so, sorry for my english). I have mailboxes using accents (eg: "Santé").
MailHeaders can be retrieved from this MailBox but moving emails to it fail.
I suggest to encode the name the same way it's done in fetching headers.
It's my first proposal in GitHub so I don't know if you see it, I suggest to modify in:
public void CopyMessages(IEnumerable uids, string destination, string mailbox = null) {
The retrieval:
string response = SendCommandGetResponse(tag + "UID COPY " + set + " " +
Util.UTF7Encode(destination).QuoteString());
Looks like it works for me.

I'm French (so, sorry for my english). I have mailboxes using accents (eg: "Santé"). 
MailHeaders can be retrieved from this MailBox but moving emails to it fail.
I suggest to encode the name the same way it's done in fetching headers.
It's my first proposal in GitHub so I don't know if you see it, I suggest to modify in:
public void CopyMessages(IEnumerable<uint> uids, string destination, string mailbox = null) {
The retrieval:
string response = SendCommandGetResponse(tag + "UID COPY " + set + " " +
	Util.UTF7Encode(destination).QuoteString());
Looks like it works for me.
@jstedfast
Copy link

This looks like a duplicate of PR #113 and issue #112

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants